Add new CentriConnect component by gresrun · Pull Request #166933 · home-assistant/core

gresrun · 2026-03-31T07:27:55Z

Proposed change

Add a new component for CentriConnect/MyPropane tank monitors

Type of change

Dependency upgrade
Bugfix (non-breaking change which fixes an issue)
New integration (thank you!)
New feature (which adds functionality to an existing integration)
Deprecation (breaking change to happen in the future)
Breaking change (fix/feature causing existing functionality to break)
Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request: Add documentation for new CentriConnect component home-assistant.io#44414
Link to developer documentation pull request:
Link to frontend pull request:

Checklist

I understand the code I am submitting and can explain how it works.
The code change is tested and works locally.
Local tests pass. Your PR cannot be merged unless tests pass
There is no commented out code in this PR.
I have followed the development checklist
I have followed the perfect PR recommendations
The code has been formatted using Ruff (ruff format homeassistant tests)
Tests have been added to verify that the new code works.
Any generated code has been carefully reviewed for correctness and compliance with project standards.

If user exposed functionality or configuration variables are added/changed:

Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
For the updated dependencies a diff between library versions and ideally a link to the changelog/release notes is added to the PR description.

To help with the load of incoming pull requests:

I have reviewed two other open pull requests in this repository.

home-assistant

Hi @gresrun

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

home-assistant · 2026-03-31T07:28:03Z

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

Copilot

Pull request overview

This pull request adds a new CentriConnect/MyPropane component to Home Assistant that integrates tank monitoring devices through a cloud polling API. The integration provides a comprehensive sensor platform for monitoring tank levels, battery status, device location, and various diagnostics.

Changes:

Adds complete CentriConnect/MyPropane integration with config flow for device authentication
Implements 16 sensor entities with device information, tank metrics, and diagnostics
Includes coordinator for polling the CentriConnect API at 6-hour intervals
Provides diagnostics support for troubleshooting
Adds comprehensive test coverage with snapshot validation

Reviewed changes

Copilot reviewed 24 out of 30 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
homeassistant/components/centriconnect/sensor.py	Main sensor platform with 16 entity descriptions for various device metrics
homeassistant/components/centriconnect/coordinator.py	Data coordinator handling API polling and data normalization
homeassistant/components/centriconnect/config_flow.py	Configuration flow for device authentication with error handling
homeassistant/components/centriconnect/init.py	Integration setup and teardown
homeassistant/components/centriconnect/entity.py	Base entity class with device info and unique ID generation
homeassistant/components/centriconnect/diagnostics.py	Diagnostics endpoint for troubleshooting
tests/components/centriconnect/	Comprehensive test suite with snapshot tests and config flow tests
manifests/config files	Integration manifest, quality scale, icons, strings, and requirements

homeassistant/components/centriconnect/sensor.py

Added suggested display precision to various sensor descriptions for better formatting.

Stale

Copilot

Pull request overview

Copilot reviewed 24 out of 30 changed files in this pull request and generated no new comments.

joostlek

Hello!

Before the PR is reviewable, please remove the brand folder and diagnostics from the initial PR. The brands should be merged to our brands repository

gresrun · 2026-04-03T05:02:33Z

Filed Brand PR: home-assistant/brands#10070

Will add back diagnostics in a later PR

Copilot

Pull request overview

Copilot reviewed 21 out of 23 changed files in this pull request and generated 3 comments.

homeassistant/components/centriconnect/sensor.py

tests/components/centriconnect/conftest.py

Copilot

Pull request overview

Copilot reviewed 21 out of 23 changed files in this pull request and generated 1 comment.

homeassistant/components/centriconnect/config_flow.py

joostlek · 2026-04-04T13:23:52Z

homeassistant/components/centriconnect/__init__.py

+    hass: HomeAssistant, entry: CentriConnectConfigEntry
+) -> bool:
+    """Unload CentriConnect/MyPropane API integration platforms and coordinator."""
+    _LOGGER.info("Unloading CentriConnect/MyPropane API integration")


no need to log IMO

joostlek · 2026-04-04T13:24:08Z

homeassistant/components/centriconnect/config_flow.py

+        vol.Required(CONF_USERNAME): str,
+        vol.Required(CONF_DEVICE_ID): str,
+        vol.Required(CONF_PASSWORD): str,


Why do we need the device id?

joostlek · 2026-04-04T13:24:31Z

homeassistant/components/centriconnect/config_flow.py

+
+    async def async_step_user(
+        self, user_input: dict[str, Any] | None = None
+    ) -> config_entries.ConfigFlowResult:


Suggested change

) -> config_entries.ConfigFlowResult:

) -> ConfigFlowResult:

joostlek · 2026-04-04T13:25:00Z

homeassistant/components/centriconnect/config_flow.py

+    }
+
+
+class CentriConnectConfigFlow(config_entries.ConfigFlow, domain=DOMAIN):


Suggested change

class CentriConnectConfigFlow(config_entries.ConfigFlow, domain=DOMAIN):

class CentriConnectConfigFlow(ConfigFlow, domain=DOMAIN):

joostlek · 2026-04-04T13:25:39Z

homeassistant/components/centriconnect/config_flow.py

+            except CentriConnectConnectionError:
+                errors["base"] = "cannot_connect"
+            except CentriConnectTooManyRequestsError:
+                errors["base"] = "cannot_connect"
+            except CentriConnectNotFoundError:
+                errors["base"] = "invalid_auth"
+            except CentriConnectEmptyResponseError:
+                errors["base"] = "unknown"
+            except CentriConnectDecodeError:
+                errors["base"] = "unknown"


These duplicate ones can be merged

joostlek · 2026-04-04T13:29:18Z

homeassistant/components/centriconnect/sensor.py

+    CentriConnectSensorEntityDescription(
+        key=CentriConnectSensorType.LATITUDE,
+        translation_key=CentriConnectSensorType.LATITUDE,
+        native_unit_of_measurement=DEGREE,
+        state_class=SensorStateClass.MEASUREMENT,
+        entity_category=EntityCategory.DIAGNOSTIC,
+        entity_registry_enabled_default=False,
+        value_fn=lambda coord: coord.data.latitude,
+    ),
+    CentriConnectSensorEntityDescription(
+        key=CentriConnectSensorType.LONGITUDE,
+        translation_key=CentriConnectSensorType.LONGITUDE,
+        native_unit_of_measurement=DEGREE,
+        state_class=SensorStateClass.MEASUREMENT,
+        entity_category=EntityCategory.DIAGNOSTIC,
+        entity_registry_enabled_default=False,
+        value_fn=lambda coord: coord.data.longitude,
+    ),


Why isn't lat/long/altitutde a device tracker?

joostlek · 2026-04-04T13:29:42Z

homeassistant/components/centriconnect/sensor.py

+    CentriConnectSensorEntityDescription(
+        key=CentriConnectSensorType.LTE_SIGNAL_LEVEL,
+        translation_key=CentriConnectSensorType.LTE_SIGNAL_LEVEL,
+        native_unit_of_measurement=PERCENTAGE,
+        state_class=SensorStateClass.MEASUREMENT,
+        entity_category=EntityCategory.DIAGNOSTIC,
+        suggested_display_precision=0,
+        value_fn=lambda coord: (
+            # The LTE signal level is estimated based on the LTE signal strength,
+            # with -140 dBm or below being 0% and -70 dBm or above being 100%.
+            min(1.0, max(((coord.data.lte_signal_strength + 140.0) / 70.0), 0.0)) * 100
+            if coord.data.lte_signal_strength is not None
+            else None
+        ),
+    ),


Either do in library

I am not sure why we need a percentage as well

joostlek · 2026-04-04T13:30:10Z

homeassistant/components/centriconnect/sensor.py

+async def async_setup_entry(
+    hass: HomeAssistant,
+    entry: CentriConnectConfigEntry,
+    async_add_entities: AddConfigEntryEntitiesCallback,
+) -> None:
+    """Set up CentriConnect sensor entities from a config entry."""
+    async_add_entities(
+        CentriConnectSensor(entry.runtime_data, description)
+        for description in ENTITIES
+        if description.value_fn(entry.runtime_data) is not None
+    )


Have this under the entity descriptions but above the sensor

joostlek · 2026-04-04T13:30:30Z

homeassistant/components/centriconnect/sensor.py

+    CentriConnectSensorEntityDescription(
+        key=CentriConnectSensorType.BATTERY_LEVEL,
+        translation_key=CentriConnectSensorType.BATTERY_LEVEL,
+        native_unit_of_measurement=PERCENTAGE,
+        state_class=SensorStateClass.MEASUREMENT,
+        device_class=SensorDeviceClass.BATTERY,
+        entity_category=EntityCategory.DIAGNOSTIC,
+        value_fn=lambda coord: (


by using the device class, you don't need all that logic for battery icons

joostlek · 2026-04-04T13:30:46Z

homeassistant/components/centriconnect/strings.json

+          "critical_level": "Critical Level",
+          "low_level": "Low Level",


our strings are in Sentence case

Initial commit of CentriConnect component

b1b4fc2

Copilot AI review requested due to automatic review settings March 31, 2026 07:27

home-assistant bot previously requested changes Mar 31, 2026

View reviewed changes

home-assistant bot added cla-needed has-tests new-integration integration: centriconnect labels Mar 31, 2026

home-assistant bot marked this pull request as draft March 31, 2026 07:28

home-assistant bot added the quality-scale label Mar 31, 2026

gresrun mentioned this pull request Mar 31, 2026

Add documentation for new CentriConnect component home-assistant/home-assistant.io#44414

Open

8 tasks

home-assistant bot added cla-recheck cla-signed and removed cla-recheck cla-needed labels Mar 31, 2026

Copilot started reviewing on behalf of gresrun March 31, 2026 07:29 View session

Copilot AI reviewed Mar 31, 2026

View reviewed changes

homeassistant/components/centriconnect/sensor.py Show resolved Hide resolved

homeassistant/components/centriconnect/sensor.py Show resolved Hide resolved

gresrun added 2 commits March 31, 2026 01:50

Add suggested_display_precision to sensor entities

8426c88

Added suggested display precision to various sensor descriptions for better formatting.

Merge branch 'dev' into centriconnect

2c85c7d

gresrun marked this pull request as ready for review March 31, 2026 08:10

Copilot AI review requested due to automatic review settings March 31, 2026 08:10

Copilot started reviewing on behalf of gresrun March 31, 2026 08:11 View session

Copilot AI reviewed Mar 31, 2026

View reviewed changes

joostlek requested changes Apr 1, 2026

View reviewed changes

home-assistant bot marked this pull request as draft April 1, 2026 14:48

gresrun mentioned this pull request Apr 3, 2026

Add brand assets for new CentriConnect component home-assistant/brands#10070

Open

10 tasks

gresrun added 2 commits April 2, 2026 23:03

Delete homeassistant/components/centriconnect/brand/logo@2x.png

226fce3

Delete homeassistant/components/centriconnect/brand/icon.png

dff57f4

Copilot AI review requested due to automatic review settings April 3, 2026 05:03

gresrun added 2 commits April 2, 2026 23:03

Delete homeassistant/components/centriconnect/brand/icon@2x.png

7ba3659

Delete homeassistant/components/centriconnect/brand/logo.png

9cc0dcc

Copilot started reviewing on behalf of gresrun April 3, 2026 05:04 View session

gresrun added 4 commits April 2, 2026 23:04

Delete tests/components/centriconnect/test_diagnostics.py

17e03ae

Delete tests/components/centriconnect/snapshots/test_diagnostics.ambr

2ec37d9

Delete homeassistant/components/centriconnect/diagnostics.py

12f2a40

Update diagnostics status from 'done' to 'todo'

d569fc7

Will add back diagnostics in a later PR

Copilot AI reviewed Apr 3, 2026

View reviewed changes

homeassistant/components/centriconnect/sensor.py Show resolved Hide resolved

tests/components/centriconnect/conftest.py Show resolved Hide resolved

tests/components/centriconnect/conftest.py Show resolved Hide resolved

Merge branch 'dev' into centriconnect

323263c

gresrun marked this pull request as ready for review April 3, 2026 05:29

Copilot AI review requested due to automatic review settings April 3, 2026 05:29

home-assistant bot requested a review from joostlek April 3, 2026 05:29

Copilot started reviewing on behalf of gresrun April 3, 2026 05:30 View session

Copilot AI reviewed Apr 3, 2026

View reviewed changes

homeassistant/components/centriconnect/config_flow.py Show resolved Hide resolved

joostlek requested changes Apr 4, 2026

View reviewed changes

home-assistant bot marked this pull request as draft April 4, 2026 13:30

		}


		class CentriConnectConfigFlow(config_entries.ConfigFlow, domain=DOMAIN):

	class CentriConnectConfigFlow(config_entries.ConfigFlow, domain=DOMAIN):
	class CentriConnectConfigFlow(ConfigFlow, domain=DOMAIN):

		"critical_level": "Critical Level",
		"low_level": "Low Level",

Uh oh!

Conversation

gresrun commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed change

Type of change

Additional information

Checklist

Uh oh!

home-assistant bot left a comment

Choose a reason for hiding this comment

Uh oh!

home-assistant bot commented Mar 31, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

joostlek left a comment

Choose a reason for hiding this comment

Uh oh!

gresrun commented Apr 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

gresrun commented Mar 31, 2026 •

edited

Loading